Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix function calling, more verbose logging, OpenAI upgrade #280

Closed
wants to merge 109 commits into from
Closed

Fix function calling, more verbose logging, OpenAI upgrade #280

wants to merge 109 commits into from

Conversation

dmytropolityka
Copy link
Contributor

@dmytropolityka dmytropolityka commented May 20, 2024 •
edited
Loading

Motivation and Context

Langchain release a new major upgrade. The version in module programming llm was far behind and had to be upgraded. This PR addresses this and also adds a bit better logging.

Description

What was changed:

  1. The way llm chains are built - was updated to use the new features of langchain.
  2. The way LLM models are initialized - because of stuff removed from the long chain.
  3. A flag to support more verbose logging.
  4. Repeated LLM invocations - in case of failures, additional invocations will be issued, up to 3 times.

Steps for Testing

Just deploy it to the test server and request feedback and suggestions from Athena and Artemis.

dmytropolityka and others added 22 commits May 12, 2024 17:34
@dmytropolityka dmytropolityka changed the title Chore/upgrade module programming llm Fix function calling, more verbose logging, OpenAI upgrade May 27, 2024
@dmytropolityka dmytropolityka marked this pull request as ready for review May 27, 2024 03:48
@dmytropolityka dmytropolityka added the deploy:athena-test1 Athena Test Server 1 label Aug 12, 2024
@dmytropolityka dmytropolityka temporarily deployed to athena-test1.ase.cit.tum.de August 12, 2024 03:46 — with GitHub Actions Inactive
@github-actions github-actions bot added lock:athena-test1 Is currently deployed to Athena Test Server 1 and removed deploy:athena-test1 Athena Test Server 1 labels Aug 12, 2024
@dmytropolityka dmytropolityka added deploy:athena-test1 Athena Test Server 1 and removed lock:athena-test1 Is currently deployed to Athena Test Server 1 labels Aug 12, 2024
@dmytropolityka dmytropolityka temporarily deployed to athena-test1.ase.cit.tum.de August 12, 2024 03:55 — with GitHub Actions Inactive
@github-actions github-actions bot added lock:athena-test1 Is currently deployed to Athena Test Server 1 and removed deploy:athena-test1 Athena Test Server 1 labels Aug 12, 2024
@maximiliansoelch maximiliansoelch added deploy:athena-test1 Athena Test Server 1 and removed lock:athena-test1 Is currently deployed to Athena Test Server 1 labels Aug 13, 2024
@maximiliansoelch maximiliansoelch temporarily deployed to athena-test1.ase.cit.tum.de August 13, 2024 09:31 — with GitHub Actions Inactive
@github-actions github-actions bot added lock:athena-test1 Is currently deployed to Athena Test Server 1 and removed deploy:athena-test1 Athena Test Server 1 labels Aug 13, 2024
@FelixTJDietrich FelixTJDietrich removed the lock:athena-test1 Is currently deployed to Athena Test Server 1 label Sep 2, 2024
Base automatically changed from feature/change-build-process to develop September 4, 2024 11:56
@dmytropolityka
Copy link
Contributor Author

Handled here: #342

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FelixTJDietrich FelixTJDietrich Awaiting requested review from FelixTJDietrich

Assignees

@dmytropolityka dmytropolityka

Labels
athena package dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dmytropolityka @FelixTJDietrich @maximiliansoelch @EneaGore